Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ignore already existing required z-nodes in global zk #202

Merged
merged 1 commit into from
Feb 13, 2017
Merged

Ignore already existing required z-nodes in global zk #202

merged 1 commit into from
Feb 13, 2017

Conversation

merlimat
Copy link
Contributor

Motivation

Following the instructions at https://github.com/yahoo/pulsar/blob/master/docs/ClusterSetup.md there is a failure when setting up additional clusters as some z-node already exist in global zookeeper.

Modifications

Ignore NodeExistsException for these z-nodes.

@merlimat merlimat added the type/bug The PR fixed a bug or issue reported a bug label Feb 11, 2017
@merlimat merlimat added this to the 1.17 milestone Feb 11, 2017
@merlimat merlimat self-assigned this Feb 11, 2017
rdhabalia
rdhabalia approved these changes Feb 13, 2017
View reviewed changes
Copy link
Contributor

@rdhabalia rdhabalia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@rdhabalia rdhabalia merged commit 60ec362 into apache:master Feb 13, 2017
sijie pushed a commit to sijie/pulsar that referenced this pull request Mar 4, 2018
@jerrypeng @sijie
checking synchronization and adding comments (apache#202)
8df9d54
hrsakai pushed a commit to hrsakai/pulsar that referenced this pull request Dec 10, 2020
@wolfstudy
[Issue: 201] Expose GetDeliveryCount method from Message interface (a…
2f57a6f 
…pache#202)

* Expose GetDeliveryCount method from Message interface

Signed-off-by: xiaolong.ran <rxl@apache.org>

* fix ci error

Signed-off-by: xiaolong.ran <rxl@apache.org>

* fix comments

Signed-off-by: xiaolong.ran <rxl@apache.org>
hangc0276 pushed a commit to hangc0276/pulsar that referenced this pull request May 26, 2021
@dockerzhang
fix fetch offset when topic is null (apache#203)
a1c8862 
bug fix for apache#202
@xiaotongwang1 xiaotongwang1 mentioned this pull request Aug 4, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rdhabalia rdhabalia rdhabalia approved these changes

Assignees

@merlimat merlimat

Labels
type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Milestone
1.17
Development

Successfully merging this pull request may close these issues.
2 participants
@merlimat @rdhabalia